Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add coverage badge #13

Merged
merged 3 commits into from
Oct 17, 2023
Merged

Add coverage badge #13

merged 3 commits into from
Oct 17, 2023

Conversation

BenediktBurger
Copy link
Member

@BenediktBurger BenediktBurger commented Oct 12, 2023

Add a coverage badge to the readme file.

Note regarding implementation:

@github-actions
Copy link

github-actions bot commented Oct 12, 2023

PyLECO Coverage

Coverage Report
FileStmtsMissCoverMissing
pyleco
   test.py61610%25–128
pyleco/core
   internal_protocols.py12120%25–70
   message.py62298%121, 148
TOTAL2117568% 

Coverage Summary

Tests Skipped Failures Errors Time
84 0 💤 0 ❌ 0 🔥 0.603s ⏱️

@BenediktBurger BenediktBurger added documentation Improvements or additions to documentation enhancement New feature or request labels Oct 12, 2023
Move the badge to the end of the paragraph.
Adding an overview, where to find the working code (other branch)
@BenediktBurger
Copy link
Member Author

@bklebel @bilderbuchi

Would you mind reviewing this small PR, such that links to the working versions are in the readme of the main version?

Copy link
Collaborator

@bklebel bklebel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, I have added a miniscule change to the wording in the README.md, which is of no great consequence, use it or leave it at your own discretion :)

README.md Outdated Show resolved Hide resolved
Co-authored-by: Benjamin Klebel-Knobloch <[email protected]>
@BenediktBurger BenediktBurger merged commit b7661d7 into main Oct 17, 2023
9 checks passed
@BenediktBurger BenediktBurger deleted the coverage-badge branch October 17, 2023 12:13
@BenediktBurger
Copy link
Member Author

Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants